Skip to content

Conversation

@jeffng-or
Copy link
Contributor

Merge of #3106 and part of #3108.

  • Fixes CTS_BUF_DISTANCE flow variable handling
  • Fixed call to genMetrics.py in AutoTuner

@luarss , @vvbandeira , @arthurjolo FYI.

Removed sky130hd gcd from sample iteration test

Signed-off-by: Jeff Ng <[email protected]>
@jeffng-or jeffng-or force-pushed the at-cts-arg-handling-1 branch from 5ad987b to 6836b18 Compare April 28, 2025 23:29
@jeffng-or
Copy link
Contributor Author

@luarss , @vvbandeira FYI. I removed sky130hd gcd from the sample iteration test because it continually fails with sample == 1. The AT can't find a successful run with only one sample, so it's a fragile test.

@luarss
Copy link
Contributor

luarss commented Apr 29, 2025

Yep that is fine - I have pushed a fix in #3025 to be more tolerant of parameter tuning failure (error code 16), so that is a more long term fix

@jeffng-or
Copy link
Contributor Author

@luarss , @vvbandeira FYI. I removed sky130hd gcd from the sample iteration test because it continually fails with sample == 1. The AT can't find a successful run with only one sample, so it's a fragile test.

It looks like the failure is a seg fault in DPL. I'm able to repro locally, so will file an issue. Closing this PR and will send up a separate PR for the genMetrics.py fix, so that we can get the AT working again.

@jeffng-or jeffng-or closed this Apr 29, 2025
@jeffng-or jeffng-or deleted the at-cts-arg-handling-1 branch May 9, 2025 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants